Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Establish atix main #1

Open
wants to merge 24 commits into
base: atix-main
Choose a base branch
from
Open

Establish atix main #1

wants to merge 24 commits into from

Conversation

sbernhard
Copy link
Member

No description provided.

@sbernhard sbernhard force-pushed the establish_atix_main branch 5 times, most recently from 36239ee to 209bcf1 Compare August 1, 2024 13:40
Matthias Dellweg and others added 20 commits August 2, 2024 17:12
@sbernhard
[ATIX] Generate enabled_repos from repofile
5abc786
@m-bucher @sbernhard
[ATIX] Fix report enabled deb-repositories only
90a4243
@sbernhard
[ATIX] Allow syspurpose file to be written
db6bcdc
@hstct @sbernhard
[ATIX] Add handling of params in repository urls
4f6026d
@goarsna @sbernhard
[ATIX] Use repofile from rhsm subpackage
e86d256
@goarsna @sbernhard
[ATIX] Remove failing method annotations
3f9ba06
@goarsna @sbernhard
[ATIX] Adjust katello script for Python3
2aed02b
@quba42 @sbernhard
[ATIX] Fix APT component selection
850ccc8
@m-bucher @sbernhard
[ATIX] Fix failing profile upload with SCA enabled
f408516 
Fixes OR-4461

Co-Authored-by: Bernhard Suttner <[email protected]>
@m-bucher @sbernhard
[ATIX] Add deb repo-setting Signed-By
f79cf5f 
Fixes OR-4769
@sbernhard
WIP Fix setup.py style
8c8167a
@sbernhard
WIP Fix baseurl style in profile
c871278
@sbernhard
WIP Fix debprofile style in profile
09c01db
@sbernhard
WIP Fix repofile style
1c6f311
@sbernhard
WIP Fix cache style
ba92938
@sbernhard
WIP Fix syspurpose style
7ca91fb
@sbernhard
WIP - fix tests. repofile moved from sub-man to rhsm
0f6741a
@sbernhard
WIP: fix repolib tests for apt. mock apt
085b20e
@sbernhard
WIP: fix repofile.py - the /etc/apt/trusted.gpg.d dir may not exist
a15dad4
@sbernhard
WIP: setup.py package profile update packaged twice
752175a
@sbernhard sbernhard force-pushed the establish_atix_main branch from eb18d10 to 752175a Compare August 2, 2024 15:46
m-bucher
m-bucher reviewed Aug 2, 2024
View reviewed changes
src/rhsm/profile.py Outdated
self._enabled_repos: EnabledRepos = EnabledRepos(repo_file)
self._content = []
for repo_file_cls, _ in get_repo_file_classes():
repo_file = repo_file_cls()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this not rather be:
repo_file = repo_file_cls(repo_file)
This might have further implications, since apparently for now this expects to always use the repo_file_cls()'s default-value for path

@sbernhard sbernhard force-pushed the establish_atix_main branch from 23d42b3 to 752175a Compare August 8, 2024 15:54
@sbernhard sbernhard force-pushed the establish_atix_main branch 2 times, most recently from a542624 to 67d9abb Compare August 8, 2024 22:40
@sbernhard sbernhard force-pushed the establish_atix_main branch from 67d9abb to 9273491 Compare August 8, 2024 23:05
@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 8, 2024

Coverage

Coverage (computed on Fedora latest) •
FileStmtsMissCoverMissing
rhsm
   profile.py2107265%28–29, 33–34, 38–39, 43–44, 50, 70, 110–114, 116–120, 122–124, 126–133, 152–154, 158–160, 198, 214–217, 262, 277, 311–315, 325–327, 331–332, 342, 353–356, 363, 368, 379–380, 394–395, 397, 400, 409–412, 425
   repofile.py48516067%35–36, 40–41, 63–64, 147–148, 171–175, 183, 229, 237, 273–279, 281, 284, 293–298, 376, 386–387, 405, 441–446, 449–456, 474–476, 478, 481, 490–492, 496–497, 519, 531–532, 534, 538, 545–546, 562–565, 612–613, 615–618, 625, 633, 640, 650–651, 673–674, 676–677, 687–688, 712–715, 719–720, 727–738, 741–749, 751–757, 761–762, 765–766, 769–770, 773–774, 776, 779–780, 782, 784–786, 788–790, 792, 794–806, 808, 811, 813, 818, 821–822, 828
subscription_manager
   cache.py62811581%30–34, 75, 82, 90, 96, 101–103, 124, 130–132, 145–148, 151, 199, 201, 238, 244–247, 254–257, 274, 277–279, 297, 306–307, 309, 318, 358–359, 393–394, 425, 427–428, 430, 443, 492, 496, 501, 506, 516, 519–520, 541, 595, 598, 623, 728, 746, 781, 783–784, 813, 816–822, 825, 857–860, 862, 873, 891, 913–914, 954–957, 959, 987, 1016–1017, 1044–1045, 1068–1069, 1073, 1077, 1079, 1087, 1129, 1157–1163, 1165, 1167, 1174–1177, 1179
   repolib.py3725684%41, 43–47, 80–85, 88, 93, 99, 130, 133, 180–181, 235–236, 246–252, 254, 399–402, 404–405, 407, 409, 411–413, 424–430, 473–476, 484–485, 492, 546, 636
   syspurposelib.py1364368%31, 33–35, 37, 96–97, 99–101, 103–105, 134–139, 176, 183, 185, 211–214, 216–221, 224–225, 229–230, 240–241, 273, 277–280
subscription_manager/cli_command
   override.py1003466%119, 121, 123–125, 131, 133, 135, 137–138, 140–143, 145–146, 148–150, 155–158, 161, 163, 166–168, 174–175, 178–180, 183
   repos.py118694%124–125, 181, 207–208, 214
TOTAL18393469974% 

Tests Skipped Failures Errors Time
2630 14 💤 0 ❌ 0 🔥 36.599s ⏱️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@m-bucher m-bucher m-bucher left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@sbernhard @m-bucher @hstct @goarsna @quba42